Skip to content

Conversation

@jbrockmendel
Copy link
Member

Waiting to update expected's until after #62801, so this won't pass yet.

@jorisvandenbossche
Copy link
Member

Should we only do this for pd.date_range for now, and not yet for pd.timedelta_range, since in general timedelta still defaults to nanoseconds? (eg when converting strings or integers in pd.to_timedelta)

@jbrockmendel
Copy link
Member Author

I think best case would be to do it all at once (xref #63018 gets everything but the string-parsing inference for td64). But if it'll make a difference in getting a release out soon, I'll happily revert.

@jorisvandenbossche
Copy link
Member

Ah, I hadn't seen #63018. Yes, my preference would also be to do it all at once. Will try to take a look at that one then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ENH/BUG: pd.date_range() still defaults to nanosecond resolution

2 participants